Квашнина Ульяна by Uliana1997 · Pull Request #25 · urfu-2016/markup-task-7

Uliana1997 · 2016-11-21T15:38:18Z

Посмотреть решение

honest-hrundel · 2016-11-21T15:38:57Z

🍅 Не пройден линтинг или базовые тесты

honest-hrundel · 2016-11-21T16:16:13Z

🍅 Не пройден линтинг или базовые тесты

Fix

honest-hrundel · 2016-11-21T16:31:57Z

🍅 Не пройден линтинг или базовые тесты

fixing

honest-hrundel · 2016-11-21T16:43:16Z

🍅 Не пройден линтинг или базовые тесты

воов

honest-hrundel · 2016-11-21T17:00:51Z

🍅 Не пройден линтинг или базовые тесты

ву

honest-hrundel · 2016-11-21T17:12:42Z

🍅 Не пройден линтинг или базовые тесты

lastt

honest-hrundel · 2016-11-21T17:31:39Z

🍅 Не пройден линтинг или базовые тесты

yep

honest-hrundel · 2016-11-21T17:37:45Z

🍏 Пройден линтинг и базовые тесты

Mokoshka · 2016-11-23T18:04:16Z

index.css

+    left: 0;
+    bottom: 20%;
+    background: #a9a9a9;
+    height: 20%;


Зачем в %?

Mokoshka · 2016-11-23T18:05:16Z

index.css

+    flex-wrap: wrap;
+    flex-direction: row;
+    justify-content: center;
+    max-width: 950px;


Зачем?

Mokoshka · 2016-11-23T18:07:39Z

index.css

+
+.breed
+{
+    width: 80%;


Зачем?

Mokoshka · 2016-11-23T18:08:36Z

index.css

+    height: 40px;
+    background-color: #a9a9a9;
+    margin: 5px;
+    padding-left: 30px;


Непонятно зачем этот отступ, выглядит так себе

Mokoshka · 2016-11-23T18:15:43Z

index.css

+
+.footer
+{
+    position: fixed;


И это тоже выглядит не очень

Mokoshka · 2016-11-23T18:16:04Z

index.css

+    background-color: #a9a9a9;
+    width: 100%;
+    bottom: 0;
+    order: 4;


Зачем?

Mokoshka · 2016-11-23T18:20:05Z

index.html

+            <label class="breed" for="abis"><span>Абиссинская</span>
+            <input name="breed hide" type="radio" id="abis"></label>
+        </div>
+        <div class="foto">


Так себе фото. Стоит переименовать

Mokoshka · 2016-11-23T18:26:18Z

index.html

+            <input name="breed hide" type="radio" id="abis"></label>
+        </div>
+        <div class="foto">
+            <div class="ashera1 name" title="Ксюша, 2 года">


И здесь name как-то тоже не совпадает с тем что внутри.
И почему ashera1?

Mokoshka · 2016-11-23T18:33:00Z

В FF все немножечко плохо
Верстке не хватает аккуратности, где-то отступов нет, где-то наоборот они слишком большие
Между котиками должны быть отступы, и границы должны находиться на одной вертикальной линии
Задание выполнено не до конца, выбор пород не работает. И сразу же потребую: у невыбранных котиков не должно показываться описание при наведении на них

Mokoshka · 2016-11-23T18:33:20Z

🍅

honest-hrundel · 2016-11-26T11:41:23Z

🍅 Не пройден линтинг или базовые тесты

change img and id selectors

honest-hrundel · 2016-11-26T11:48:10Z

🍏 Пройден линтинг и базовые тесты

Uliana1997 · 2016-11-26T11:49:40Z

🍏

mokhov · 2016-11-26T19:08:15Z

Ещё раз увижу, что перед дедлайном отправляется не рабочее решение – будут санкции!

honest-hrundel · 2016-11-26T20:01:09Z

🍏 Пройден линтинг и базовые тесты

Mokoshka · 2016-11-27T16:44:24Z

🚀

forshtreter · 2016-11-28T15:09:55Z

https://yadi.sk/i/fXzO4FUuzgRhM ой. При рефреше пропадает, но надо починить всё равно.
Ещё хочется чтобы названия пород тоже как-то подсвечивались при выборе, а то непонятно что выбрал

forshtreter · 2016-11-28T15:11:45Z

🍅

Uliana1997 added 2 commits November 21, 2016 18:09

Add files via upload

93bc834

Add files via upload

7b00538

Квашнина Ульяна

d96ae9f

Квашнина Ульяна

8fad2a1

Fix

Квашнина Ульяна

44b082f

fixing

Квашнина Ульяна

44624d4

воов

Квашнина Ульяна

d762d83

ву

Квашнина Ульяна

a6ab910

lastt

Квашнина Ульяна

252ad4f

yep

honest-hrundel assigned Mokoshka Nov 21, 2016

Mokoshka reviewed Nov 23, 2016

View reviewed changes

index.css Outdated

left: 0;

bottom: 20%;

background: #a9a9a9;

height: 20%;

Copy link

Mokoshka Nov 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем в %?

Mokoshka reviewed Nov 23, 2016

View reviewed changes

index.css Outdated

flex-wrap: wrap;

flex-direction: row;

justify-content: center;

max-width: 950px;

Copy link

Mokoshka Nov 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем?

Mokoshka reviewed Nov 23, 2016

View reviewed changes

index.css Outdated

.breed

{

width: 80%;

Copy link

Mokoshka Nov 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем?

Mokoshka reviewed Nov 23, 2016

View reviewed changes

index.css Outdated

.footer

{

position: fixed;

Copy link

Mokoshka Nov 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И это тоже выглядит не очень

Mokoshka reviewed Nov 23, 2016

View reviewed changes

index.css Outdated

background-color: #a9a9a9;

width: 100%;

bottom: 0;

order: 4;

Copy link

Mokoshka Nov 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем?

Mokoshka reviewed Nov 23, 2016

View reviewed changes

Квашнина Ульяна

2bf2a47

Квашнина Ульяна

b667fc1

change img and id selectors

Квашнина Ульяна

fbf78eb

honest-hrundel assigned forshtreter and unassigned Mokoshka Nov 27, 2016

Comments

Conversation

Uliana1997 commented Nov 21, 2016 • edited by honest-hrundel Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

honest-hrundel commented Nov 21, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Mokoshka commented Nov 23, 2016

Uh oh!

Mokoshka commented Nov 23, 2016

Uh oh!

honest-hrundel commented Nov 26, 2016

Uh oh!

honest-hrundel commented Nov 26, 2016

Uh oh!

Uliana1997 commented Nov 26, 2016

Uh oh!

mokhov commented Nov 26, 2016

Uh oh!

honest-hrundel commented Nov 26, 2016

Uh oh!

Mokoshka commented Nov 27, 2016

Uh oh!

forshtreter commented Nov 28, 2016

Uh oh!

forshtreter commented Nov 28, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uliana1997 commented Nov 21, 2016 •

edited by honest-hrundel

Loading